Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add new CVar mp_free_armor #609

Merged
merged 12 commits into from
Jun 22, 2021
Merged

Add new CVar mp_free_armor #609

merged 12 commits into from
Jun 22, 2021

Conversation

AlexSwartz
Copy link
Contributor

@AlexSwartz AlexSwartz commented Mar 23, 2021 •
edited by wopox1337
Loading

Resolves #560

  • Implement the free_armor logic.
  • Update game.cfg with mp_free_armor command
  • Update readme with mp_free_armor command
  • Register mp_free_armor cvar
// Give the player free armor on player spawn
// 0 - No armor (default behavior)
// 1 - Give Kevlar
// 2 - Give Kevlar and Helmet
//
// Default value: "0"
mp_free_armor 0

@AlexSwartz
Implement the free_armor logic.
24c183c 
Update game.cfg with mp_free_armor command
Update readme with mp_free_armor command
Register mp_free_armor cvar
@etojuice
Copy link
Contributor

Fix spaces.
Move the code that gives armor to OnSpawnEquip.

@AlexSwartz
Moved mp_free_armor to OnSpawnEquip
1b07fea 
Moved to OnSpawnEquip
Fixed spacing
@etojuice
Copy link
Contributor

If you want to replicate the feature from CS:GO, then you should know that in CS:GO players receive free armor regardless of being dead before spawning. In other words, give armor outside of GiveDefaultItems condition.

@wopox1337 wopox1337 self-requested a review June 21, 2021 09:33
@wopox1337 wopox1337 self-assigned this Jun 21, 2021
@wopox1337 wopox1337 requested a review from s1lentq June 21, 2021 10:28
@wopox1337 wopox1337 changed the title Add feature mp_free_armor 0/1/2 Add new CVar mp_free_armor Jun 21, 2021
@wopox1337 wopox1337 merged commit 191857d into s1lentq:master Jun 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wopox1337 wopox1337 wopox1337 approved these changes

@s1lentq s1lentq Awaiting requested review from s1lentq

Assignees

@wopox1337 wopox1337

@AlexSwartz AlexSwartz

Labels
Priority: Low Status: Completed Type: Enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feature request] mp_free_armor
3 participants
@AlexSwartz @etojuice @wopox1337